Arm32: correct string.h functions for "int" -> "unsigned char" conversion
authorJan Beulich <jbeulich@suse.com>
Wed, 24 Aug 2022 12:33:06 +0000 (14:33 +0200)
committerJulien Grall <jgrall@amazon.com>
Thu, 25 Aug 2022 14:35:21 +0000 (15:35 +0100)
commitcbb35e72802f3a285c382a995ef647b59e5caf2f
treee6e848197f1f4558ae383a7c1266b66c8b2ba1b0
parentc3bd0b83ea5b7c0da6542687436042eeea1e7909
Arm32: correct string.h functions for "int" -> "unsigned char" conversion

While Arm64 does so uniformly, for Arm32 only strchr() currently handles
this properly. Add the necessary conversion also to strrchr(), memchr(),
and memset().

As to the placement in memset(): Putting the new insn at the beginning
of the function is apparently deemed more "obvious". It could be placed
later, as the code reachable without ever making it to the "1" label
only ever does byte stores.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com>
xen/arch/arm/arm32/lib/memchr.S
xen/arch/arm/arm32/lib/memset.S
xen/arch/arm/arm32/lib/strrchr.S